docs: replace some usage of "h-series" #563
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
CalMacCQ
commented
Jan 15, 2025
| @@ -102,7 +102,7 @@ the circuit 2-qubit gate count is not reduced after compilation. | |||
| * - [AutoRebase [2]](inv:#*.AutoRebase) | |||
| - [SynthesiseTket](inv:#*.SynthesiseTket) | |||
| - [FullPeepholeOptimise [3]](inv:#*.passes.FullPeepholeOptimise) | |||
| - [RemoveBarriers](inv:#*pytket.passes.RemoveBarriers) | |||
| - [RemoveBarriers](inv:#*pytket._tket.passes.RemoveBarriers) | |||
There was a problem hiding this comment.
This is a separate fix required to resolve a sphinx warning. Looks weird to have the _tket in there but have verfied that it'll resolve correctly in the deployed site.
irfankhan10
requested changes
Jan 21, 2025
docs/api.md
Outdated
| @@ -1,6 +1,6 @@ | |||
| # API documentation | |||
|
|
|||
| The pytket-quantinuum extension allows submission of pytket circuits to the H-series trapped ion devices (and emulators) via the {py:class}`QuantinuumBackend`. | |||
| The pytket-quantinuum extension allows submission of pytket circuits to the Quantinuum trapped ion devices (and emulators) via the {py:class}`QuantinuumBackend`. | |||
There was a problem hiding this comment.
Replace "the Quantinuum trapped ion devices" with "Quantinuum Systems"
docs/index.md
Outdated
| @@ -14,7 +14,7 @@ and Windows. To install, run: | |||
| pip install pytket-quantinuum | |||
| ``` | |||
|
|
|||
| For example usage see the [pytket-quantinuum section](https://docs.quantinuum.com/h-series/trainings/getting_started/pytket_quantinuum/pytket_quantinuum.html) of the H-series documentation. | |||
| For example usage see the [pytket-quantinuum section](https://docs.quantinuum.com/h-series/trainings/getting_started/pytket_quantinuum/pytket_quantinuum.html) of the Quantinuum hardware documentation. | |||
irfankhan10
approved these changes
Jan 23, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Theres not much usage of h-series to update. For the two mentions in the docs I've used "Quantinuum" and "Quantinuum hardware" which I think is clear enough.
There was also some usage in the calendar methods of
QuantinuumBackend. I just removed the usage here. Fine with doing something else instead though. I'm not too worried about these docstrings.There are some
/h-serieslinks that remain that crosslink to other pages of the docs site. We'll have to update the url before these can be changed unless we can get it to work with relative linking.